update - #431

…d loss)

ETH-family block headers carry no confirmations/topoheight, and the daemon returns
non-canonical blocks by hash (verified against the live ETC daemon: getBlockByHash of an
uncle / non-canonical hash returns the block). So an orphaned found block -- not canonical
and not referenced as an uncle within the inclusion window -- came back from
getAnyBlockHeaderByHash non-null with error=null: neither checkOrphans (mo_altblockmanager)
nor isAltblockOrphanResponse (block_manager) detected it (both key off confirmations /
topoheight === -1), and the height-fallback is gated behind a non-null error. The orphan
was then traded to XMR and paid to miners for a block that earned nothing on-chain.

Mark the true-orphan fallthrough with confirmations = -1, the sentinel both consumers
already recognise, so it is invalidated instead of paid. The canonical (hash === canonical)
and uncle (createEthUncleReward) branches return earlier and are untouched, so valid blocks
and uncles are never misflagged (ETC uncle-inclusion distance <= 7 is inside the +-8 scan
window). Verified against the live ETC daemon; full suite green.

Follow-up to the ETC-orphan fix, after a same-class sweep + live-daemon verification:

- ERG (factories.js createErgRpc.getAnyBlockHeaderByHash): like ETC, the Ergo node serves
  forked/orphaned blocks by id with HTTP 200 and no orphan marker, and this handler had no
  canonical comparison at all, so an orphaned ERG found block returned as a clean success and
  was paid. Now compares against the single canonical id at the block's height and marks a
  non-canonical block confirmations=-1; ambiguous lookups fall through to the valid path.

- Cryptonote alts (xla/ryo/zeph/sal/arq/ccx + XMR alt blocks): monerod get_block_header_by_hash
  returns an orphaned block non-null with orphan_status=true (verified vs the live XMR daemon:
  13 alt chains present; alt header orphan_status=true, main=false), but isAltblockOrphanResponse
  only checked topoheight/confirmations/error -> the orphan passed and was paid. Added
  orphan_status===true to the check.

Valid blocks unaffected (orphan_status=false; ERG marks only a confirmed-different canonical id).
Full suite green.

mom v0.7.0 benchmarks the active GPU algos (autolykos2/etchash/kawpow/pearl) at startup by
default (bench_algo_params=1); those aren't in MOM_NO_BENCH_ALGOS, so the "no-bench" live
config still ran slow (and on some Intel Xe GPUs unstable) DAG benchmarks before mining. Set
bench_algo_params=0 to skip benchmarking and go straight to the pool job.

…drop an accepted block

submitEthBlock confirms acceptance on eth_submitWork===true, then resolveEthSubmittedBlockHash
waited 30s and did a SINGLE ethBlockFind to discover the on-chain hash. If the daemon was
down/slow/restarting in that window, ethBlockFind/ethBlockCheck return null, the function
returned the zero-hash, and share_blocks dropped the already-accepted (on-chain) block
uncredited (admin FYI only, no retry). Compounds with monitorNodes' daemon restarts.

ethBlockFind is read-only/idempotent, so retry it a bounded number of times
(ETH_BLOCK_RESOLVE_ATTEMPTS=10 x 30s) before falling back to the zero-hash. Single-attempt
success is unchanged.

Limits (option A, in-memory): retry state is not durable, so a pool restart inside the window
still loses the in-flight block; outages longer than ~5 min still fall back to the zero-hash
drop; a genuine orphan is dropped after the retries instead of immediately. A durable fix
(persist the pending resolution) was deferred. + tests (transient-recover, exhausted-fallback).
Suite green.

…ropping a refind (R14, partial)

Main blocks are keyed by height, so saveResolvedBlock treated ANY existing blockDB[height] entry
as a duplicate (hash-blind). After a reorg orphans block A at height N (unlocker sets valid=false)
and the pool finds canonical block C at N, saveResolvedBlock saw the slot occupied and dropped C
-> lost block, miners unpaid.

Now: same hash -> genuine duplicate (unchanged); different hash but the recorded block was
orphaned (valid===false) -> replace the orphan tombstone with the new valid block; a different
still-valid block -> still "duplicate" (cannot hold two at one height key). Normal one-block-per-
height operation is unchanged. The remaining two-still-valid-at-one-height window needs the full
fix (height in the Block value + forward-probe, like AltBlock) -- tracked as R14 option 1. Suite green.

…c236fe / b31b6ed)

Adds the coverage the cons-review flagged as missing for three money-path fixes:
- basics.js: eth-family getAnyBlockHeaderByHash tags a true orphan (not canonical, no uncle
  reference) with confirmations=-1; erg getAnyBlockHeaderByHash tags a true orphan (a
  different canonical id occupies the height) with confirmations=-1.
- block_jobs.js: saveResolvedBlock replaces an orphaned (valid=false) same-height block with
  the canonical refind, and keeps an existing still-valid block (treated as duplicate).
Tests-only; no production code change.

… (safe-additive audit fixes)

The workerMax check pre-incremented (++count <= workerMax), so an over-limit login that is rejected
still bumped the count; that worker never connects, so it is never decremented and the count drifts
up over time until legitimate miners under the real limit are wrongly rejected. Check before
incrementing (intent of refactor ef46e97 was only to bound worker count). Suite green.

…unt leak, long_runner key-0 resume, template one-shot

- manage_scripts/exchange_recovery_cache_common.js: key buildBlockLookup by the LMDB `_key` the
  callback yields, not the non-existent block.id (was always NaN -> lookup 100% broken).
- lib/pool/miner_registry.js: check proxy worker count before incrementing so a rejected over-limit
  login doesn't leak the count (twin of the protocol.js 302d649 fix).
- lib/long_runner.js: compare goToRange() !== false in the reverse-resume branch so an LMDB key
  value of 0 isn't mis-treated as "not found" (mirrors the forward branch).
- lib/pool/templates.js: make the header-fetch-error reschedule one-shot (requireTrue) like the
  success path, so a bootstrap (undefined repeating) call can't spawn a second poll loop.
- factories.js + protocol.js: fix 'extranoces' -> 'extranonces' miner-facing typo (+ test).
Suite green (504 pass).

…r guard/import, collapse duplicate coinHashFactor branches (behavior-preserving, eslint+tests green)

…(skip cycle, not crash)

createBlockTemplate (RtmBlockTemplate/RavenBlockTemplate) runs inside the daemon-RPC reply callback,
which jsonRequest invokes OUTSIDE its try/catch — so any template-build throw (malformed bits/version/
previousblockhash, too-many/unparseable txs) propagated uncaught and could crash the process. Wrap the
build in try/catch and, on failure, log + return like a no-result reply so the pool skips that template
cycle and retries next poll. Suite 504/504.

Recognize the current lin, mac, and win release names while retaining compatibility with legacy platform suffixes. Cover platform selection and Linux compatibility-build preference in the live helper tests.